process: add initial tool management #932

masc2023 · 2025-04-15T06:48:30Z

Resolves: #858

github-actions · 2025-04-15T06:49:17Z

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ✅ Passed

Click to expand output

[License Check Output]
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
INFO: Invocation ID: f3ec30aa-59ad-46b7-adf4-48cc2a39cda9
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (90 packages loaded, 10 targets configured)

Analyzing: target //:license-check (98 packages loaded, 10 targets configured)

Analyzing: target //:license-check (98 packages loaded, 10 targets configured)

Analyzing: target //:license-check (131 packages loaded, 821 targets configured)

Analyzing: target //:license-check (134 packages loaded, 855 targets configured)

Analyzing: target //:license-check (145 packages loaded, 2451 targets configured)

Analyzing: target //:license-check (145 packages loaded, 2465 targets configured)

Analyzing: target //:license-check (149 packages loaded, 4611 targets configured)

INFO: Analyzed target //:license-check (150 packages loaded, 4737 targets configured).
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 18.170s, Critical Path: 0.42s
INFO: 13 processes: 4 disk cache hit, 9 internal.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt -review -project automotive.score -repo https://github.com/eclipse-score/score -token otyhZ4eaRYK1tKLNNF-Y
[main] INFO Querying Eclipse Foundation for license data for 76 items.
[main] INFO Found 52 items.
[main] INFO Querying ClearlyDefined for license data for 24 items.
[main] INFO Found 24 items.
[main] INFO Vetted license information was found for all content. No further investigation is required.

github-actions · 2025-04-15T06:51:27Z

The created documentation from the pull request is available at: docu-html

aschemmel-tech

see inline comments

docs/process/process_areas/tool_management/tool_management_getstrt.rst

docs/process/process_areas/tool_management/tool_management_workflow.rst

docs/process/process_areas/tool_management/tool_management_workproducts.rst

docs/process/process_areas/tool_management/guidance/tool_management_checklist.rst

docs/process/process_areas/tool_management/guidance/tool_management_template.rst

docs/process/process_areas/tool_management/guidance/tool_management_checklist.rst

docs/platform_management_plan/tool_management.rst

pahmann · 2025-04-29T06:32:32Z

docs/platform_management_plan/tool_management.rst

+The purpose of the Tool Management Plan is to guide the identification, evaluation and qualification of
+project tools.
+
+S-CORE objectives is to use OSS tools for the generation of all required work products. Thus all


We also use GitHub as a "tool". This is not open source.

pahmann · 2025-04-29T06:36:24Z

docs/platform_management_plan/tool_management.rst

+
+**To Step 2**
+
+Based on the found malfunctions or threats judge, if these can be all detected or prevented?


This is not a question and hard to understand. Better use:

Suggested change

Based on the found malfunctions or threats judge, if these can be all detected or prevented?

Based on the malfunctions or threats found, assess whether these can be detected or prevented.

pahmann · 2025-04-29T06:41:22Z

docs/platform_management_plan/tool_management.rst

+Qualification
+^^^^^^^^^^^^^
+
+As method validation of software tool is applied.


The method used is the validation of the software tool.
Or
The method used is software tool validation.
Or
As method "validation of software tool" is applied.

Modified to match last version

pahmann · 2025-04-29T06:42:22Z

docs/platform_management_plan/tool_management.rst

+Update the Tool Verification Report accordingly to document the outcomes.
+
+For the generation of requirements, verification and analyses the processes defined in the
+:need:`wp__platform_mgmt` shall be used (Requirements Engineering, Verification,


Would it improve the quality to link to the respective parts for requirements, etc. rather than just say "read the platform management plan"?

Add concrete links, as available for now

pahmann · 2025-04-29T06:44:16Z

docs/process/process_areas/tool_management/guidance/tool_management_checklist.rst

+   :complies: std_req__iso26262__support_1141, std_req__iso26262__support_1142, std_req__iso26262__support_1143, std_req__iso26262__support_11441, std_req__iso26262__support_11442, std_req__iso26262__support_11451, std_req__iso26262__support_11452, std_req__iso26262__support_11453, std_req__iso26262__support_11454, std_req__iso26262__support_11461, std_req__iso26262__support_11462
+
+   | **1. Purpose**
+   | The purpose of this checklist is to collect the topics to be checked during a tool verification..


Two "." at the end of the sentence

Suggested change

| The purpose of this checklist is to collect the topics to be checked during a tool verification..

| The purpose of this checklist is to collect the topics to be checked during a tool verification.

removed one

pahmann · 2025-04-29T06:45:04Z

docs/process/process_areas/tool_management/guidance/tool_management_checklist.rst

+
+   | **1. Purpose**
+   | The purpose of this checklist is to collect the topics to be checked during a tool verification..
+   | It will not be filled out but considered during the review and monitoring to complete of the Tool Verification Report.


Suggested change

| It will not be filled out but considered during the review and monitoring to complete of the Tool Verification Report.

| It will not be filled out, but considered during the review and monitoring to complete of the Tool Verification Report.

pahmann · 2025-04-29T06:49:27Z

docs/process/process_areas/tool_management/guidance/tool_management_template.rst

+
+**Optional Section for Tool Qualification**
+-------------------------------------------
+based on method: validation of the software tool


Suggested change

based on method: validation of the software tool

Based on method: validation of the software tool

pahmann · 2025-04-29T06:51:37Z

docs/process/process_areas/tool_management/tool_management_concept.rst

+Key concept
+***********
+S-CORE objective is to use OSS Tools only. The selection criteria for OSS Tool therefore must
+include the determination of the Tool Confidence Level `(TCL) <https://www.validas.de/publications/automotive2012.pdf>`_


Does this mean that if a tool is not OSS, I do not need a TCL?
It says currently: S-CORE only uses OSS tools [...] therefore must include [...] TCL

No, rephrased

pahmann · 2025-04-29T06:54:19Z

docs/process/process_areas/tool_management/tool_management_concept.rst

+
+Activities for Tool Management
+******************************
+


tlm stands for tool management correct? Why not referring to as tool_mgmt_creation?

changed as this was some wrong documentation

docs/platform_management_plan/tool_management.rst

AlexanderLanin · 2025-04-30T08:32:49Z

docs/platform_management_plan/tool_management.rst

+The purpose of the Tool Management Plan is to guide the identification, evaluation and qualification of
+project tools.


Using "qualification" without a clear reference to safety is confusing?! As usually there is no qualification. Maybe just change it to "safety qualification"?

Please again, forgot looking only to safety, our process framework does conform to multiple standards and for you as user, only our process are relevant, the framework is mapped to the relevant standard and it defines, what we do for qualification, in our case. I add for current state just safe and secure use of tools, maybe replace in future by trustable use in the tool, where trustable may include more than safe and secure

docs/platform_management_plan/tool_management.rst

AlexanderLanin · 2025-04-30T08:36:36Z

docs/platform_management_plan/tool_management.rst

+Qualification
+^^^^^^^^^^^^^
+
+As method "validation of software tool" is applied.


Why not apply the other methods? I would personally much rather have a tool development process.

What development process? I just said, that we use S-CORE parts for tools too, and independent of that, having requirements or goals or how ever you call it, and you test that is minimum in my view

AlexanderLanin · 2025-04-30T08:59:17Z

docs/process/process_areas/tool_management/tool_management_getstrt.rst

+* Make familiar with the development and supporting process descriptions in :ref:`process_description`
+* Make familiar with the relevant sections of the :ref:`Platform Management Plan <pmp>`, here especially with :need:`Tool Management Plan <doc__platform_tool_management_plan>`


Suggested change

* Make familiar with the development and supporting process descriptions in :ref:`process_description`

* Make familiar with the relevant sections of the :ref:`Platform Management Plan <pmp>`, here especially with :need:`Tool Management Plan <doc__platform_tool_management_plan>`

* Make yourself familiar with the development and supporting process descriptions in :ref:`process_description`

* Make yourself familiar with the relevant sections of the :ref:`Platform Management Plan <pmp>`, here especially with :need:`Tool Management Plan <doc__platform_tool_management_plan>`

Unclear, just understood you changed the order, that is done

I've added "yourself" --> "Make yourself familiar"

docs/process/process_areas/tool_management/tool_management_workflow.rst

AlexanderLanin · 2025-04-30T09:02:41Z

docs/process/process_areas/tool_management/tool_management_workflow.rst

+
+   Finally the Tool Verification Report is verified and approved, and thus in Released state.
+
+   For creating the Tool Verification Report a template must be used.


"The defined template"?

Note however, that technically we'll most likely use markdown or even yaml and not rst files for that. However we can use the template chapters etc.

you see in the needs the linked templates, so I added the linked template, which is visible in the Sphinx HTML documentation

docs/process/process_areas/tool_management/tool_management_workproducts.rst

aschemmel-tech

my remarks are all covered

Resolves: #858

Review outdated

AlexanderLanin · 2025-04-30T15:58:44Z

@masc2023 I would highly recommend pushing new commits instead of squashing locally. I now have no chance to see what has changed since my review.
Instead I recommend squashing here on the website once the pull request is approved.

masc2023 force-pushed the masc2023_add_initial_tool_management branch from 3242aa0 to af8341c Compare April 15, 2025 09:41

masc2023 marked this pull request as ready for review April 15, 2025 09:48

masc2023 requested a review from aschemmel-tech April 15, 2025 09:48

masc2023 force-pushed the masc2023_add_initial_tool_management branch from af8341c to c0e3042 Compare April 16, 2025 07:03

masc2023 self-assigned this Apr 16, 2025

masc2023 force-pushed the masc2023_add_initial_tool_management branch from c0e3042 to f8a7453 Compare April 16, 2025 08:03

aschemmel-tech requested changes Apr 23, 2025

View reviewed changes

masc2023 force-pushed the masc2023_add_initial_tool_management branch from f8a7453 to 37c6d46 Compare April 25, 2025 10:23

pahmann reviewed Apr 29, 2025

View reviewed changes

pahmann requested changes Apr 29, 2025

View reviewed changes

masc2023 requested review from mmr1909 and MaximilianSoerenPollak April 29, 2025 08:03

masc2023 force-pushed the masc2023_add_initial_tool_management branch 2 times, most recently from ed3fa5e to 4de368a Compare April 29, 2025 10:22

MaximilianSoerenPollak mentioned this pull request Apr 30, 2025

Tooling information collection #1015

Open

dcalavrezo-qorix reviewed Apr 30, 2025

View reviewed changes

docs/platform_management_plan/tool_management.rst Outdated Show resolved Hide resolved

AlexanderLanin previously requested changes Apr 30, 2025

View reviewed changes

aschemmel-tech previously approved these changes Apr 30, 2025

View reviewed changes

proces: add initial tool management

bd7e53e

Resolves: #858

masc2023 dismissed aschemmel-tech’s stale review via bd7e53e April 30, 2025 13:55

masc2023 force-pushed the masc2023_add_initial_tool_management branch from 4de368a to bd7e53e Compare April 30, 2025 13:55

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

process: add initial tool management #932

process: add initial tool management #932

masc2023 commented Apr 15, 2025

github-actions bot commented Apr 15, 2025 •

edited

Loading

github-actions bot commented Apr 15, 2025

aschemmel-tech left a comment

pahmann Apr 29, 2025

masc2023 Apr 29, 2025

pahmann Apr 29, 2025

masc2023 Apr 29, 2025

pahmann Apr 29, 2025

masc2023 Apr 29, 2025

pahmann Apr 29, 2025

masc2023 Apr 29, 2025

pahmann Apr 29, 2025

masc2023 Apr 29, 2025

pahmann Apr 29, 2025

masc2023 Apr 29, 2025

pahmann Apr 29, 2025

masc2023 Apr 29, 2025

pahmann Apr 29, 2025

masc2023 Apr 29, 2025

pahmann Apr 29, 2025

masc2023 Apr 29, 2025

AlexanderLanin Apr 30, 2025

masc2023 Apr 30, 2025

AlexanderLanin Apr 30, 2025

masc2023 Apr 30, 2025

AlexanderLanin Apr 30, 2025

masc2023 Apr 30, 2025

AlexanderLanin Apr 30, 2025

AlexanderLanin Apr 30, 2025

masc2023 Apr 30, 2025

aschemmel-tech left a comment

AlexanderLanin commented Apr 30, 2025


		To Step 2

		Based on the found malfunctions or threats judge, if these can be all detected or prevented?

	Based on the found malfunctions or threats judge, if these can be all detected or prevented?
	Based on the malfunctions or threats found, assess whether these can be detected or prevented.

	\| The purpose of this checklist is to collect the topics to be checked during a tool verification..
	\| The purpose of this checklist is to collect the topics to be checked during a tool verification.

	\| It will not be filled out but considered during the review and monitoring to complete of the Tool Verification Report.
	\| It will not be filled out, but considered during the review and monitoring to complete of the Tool Verification Report.

	based on method: validation of the software tool
	Based on method: validation of the software tool


		Activities for Tool Management
		******************************

		The purpose of the Tool Management Plan is to guide the identification, evaluation and qualification of
		project tools.

		* Make familiar with the development and supporting process descriptions in :ref:`process_description`
		* Make familiar with the relevant sections of the :ref:`Platform Management Plan <pmp>`, here especially with :need:`Tool Management Plan <doc__platform_tool_management_plan>`


		Finally the Tool Verification Report is verified and approved, and thus in Released state.

		For creating the Tool Verification Report a template must be used.

process: add initial tool management #932

Are you sure you want to change the base?

process: add initial tool management #932

Conversation

masc2023 commented Apr 15, 2025

github-actions bot commented Apr 15, 2025 • edited Loading

License Check Results

github-actions bot commented Apr 15, 2025

aschemmel-tech left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

aschemmel-tech left a comment

Choose a reason for hiding this comment

AlexanderLanin commented Apr 30, 2025

github-actions bot commented Apr 15, 2025 •

edited

Loading